Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Formula dependency updates #63

Conversation

DrShaneBurke
Copy link

@DrShaneBurke DrShaneBurke commented Mar 6, 2021

Please consider merging these revisions with the master branch of coin-or-tools/homebrew-coinor

In this update I have:

  1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
  2. Updated version references to most recent (as of 3/6/2021)
  3. Updated sha256 checksums for appropriate versions
  4. Commented out "make" "test" as described at Error installing CoinUtils on Mac M1 coin-or/CoinUtils#151 (comment) to bypass an assertion error

Changes were related to the issue discussed at coin-or/CoinUtils#151

This was successfully compiled, without error, locally when "brew install coin-or-tools/coinor/cbc" was run within PyCharm terminal (community version 2020.3) running Python 3.9 (installed via homebrew version 3.0.4-50) on a 2020 MacBook Air with an M1 Apple Silicon chip running macOS Big Sur (version 11.2.2) and XCode version 12.4

1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version reference to most recent (2.10.5)
3. Updated sha256 for appropriate version (pending)

Changes related to the issue discussed at coin-or/CoinUtils#151
1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version reference to most recent (2.10.5)
3. Updated sha256 for appropriate version (pending)

Changes related to the issue discussed at coin-or/CoinUtils#151
1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version reference to most recent (1.17.6)
3. Updated sha256 for appropriate version (pending)

Changes related to the issue discussed at coin-or/CoinUtils#151
1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version reference to most recent (0.60.3)
3. Updated sha256 for appropriate version (pending)

Changes related to the issue discussed at coin-or/CoinUtils#151
1. Updated version reference to most recent (1.2.9)
2. Updated sha256 for appropriate version (pending)
1. Updated version reference to most recent (1.2.12)
2. Updated sha256 for appropriate version (pending)
1. Updated version reference to most recent (1.1.7)
2. Updated sha256 for appropriate version (pending)
1. Updated version reference to most recent (1.2.8)
2. Updated sha256 for appropriate version (pending)
1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version reference to most recent (2.10.5)
3. Updated sha256 for appropriate version (pending)
1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version reference to most recent (2.11.4)
3. Updated sha256 for appropriate version (pending)

Changes related to the issue discussed at coin-or/CoinUtils#151
1. Updated homepage, url, and head to refer to GitHub rather than coin-or.org
2. Updated version references to most recent (as of 3/6/2021)
3. Updated sha256 checksums for appropriate versions
4. Commented out "make" "test" as described at coin-or/CoinUtils#151 (comment) to bypass assertion error

Changes were related to the issue discussed at coin-or/CoinUtils#151

Successfully compiled without error when "brew tap coin-or-tools/coinor" then "brew install coin-or-tools/coinor/cbc" was run within PyCharm terminal (community version 2020.3) running Python 3.9 (installed via homebrew version 3.0.4-50) on a 2020 MacBook Air with an M1 Apple Silicon chip running macOS Big Sur (version 11.2.2) and XCode version 12.4
@CLAassistant
Copy link

CLAassistant commented Mar 6, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Author

@DrShaneBurke DrShaneBurke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Wasn't sure if that was how it was supposed to look, I think I definitely clicked the wrong button on GitHub when pushing it back

@tkralphs
Copy link
Member

tkralphs commented Mar 6, 2021

Thanks! I did a little cleanup, but it looks like the testing is a bit broken and it would be good to fix that up before merging. I'll have to come back to that a bit later.

@dpo
Copy link
Contributor

dpo commented Mar 5, 2022

Many thanks. With recent updates, it seems we now have the updates you submitted here, so we're going to close this.

@dpo dpo closed this Mar 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants